Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Features added #15

Merged
merged 9 commits into from
Apr 11, 2020
Merged

Features added #15

merged 9 commits into from
Apr 11, 2020

Conversation

QuickLearner171998
Copy link
Contributor

@QuickLearner171998 QuickLearner171998 commented Mar 29, 2020

Reference Issues/PRs

#11

What does this implement/fix? Explain your changes.

Following features are added:

  1. Threshold can now be set by the user.
  2. Pascal VOC saving format.
  3. Add all classes at once without selecting each manually.
  4. Option to choose between tensorflow and keras models.
  5. Display accuracy and model name.

Any other comments?

QuickLearner171998 and others added 7 commits March 30, 2020 00:57
Following features are added:
1) Threshold can now set by the user.
2) Pascal VOC saving format
3) Add all classes at once without selecting each manually
@virajmavani
Copy link
Owner

Hi @QuickLearner171998! Thank you for lending a hand in implementing these features. I haven't set-up CI yet so I will have to manually test it. Once done, I will be able to merge the changes.

Thank you!

main.py Outdated Show resolved Hide resolved
main.py Outdated Show resolved Hide resolved
main.py Outdated Show resolved Hide resolved
@virajmavani
Copy link
Owner

These are some great changes! Thank you for contributing to the project!

Once the changes have been made, I will go ahead and merge.

Later, I am planning to refactor the whole project as it is abominable. I don't know what I was thinking when I wrote it 2 years back.

If interested, we can make a Slack group for developers and chat over it so that anyone else can also join.

@QuickLearner171998
Copy link
Contributor Author

Yes, that would be great!!

@virajmavani
Copy link
Owner

Looks good enough. I will merge. Good job!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants